Skip to content

wave: document KYC requirement for issue applications#60

Merged
efstajas merged 5 commits into
mainfrom
feat/kyc-required-for-applications
May 21, 2026
Merged

wave: document KYC requirement for issue applications#60
efstajas merged 5 commits into
mainfrom
feat/kyc-required-for-applications

Conversation

@efstajas
Copy link
Copy Markdown
Contributor

@efstajas efstajas commented May 21, 2026

Summary

Companion docs for drips-network/wave#596 (gates issue applications behind verified KYC) and drips-network/app#1906 (frontend prompt + blocker).

The app links to solving-issues-and-earning-rewards#verifying-your-identity from the in-app KYC prompt, so this PR introduces that anchor and aligns surrounding copy with the new rule that KYC is required to apply, not just to withdraw.

Changes

  • wave/contributors/solving-issues-and-earning-rewards.mdx
    • New section ## Verifying your identity — the destination the app's "Learn more" link points to. Covers why, how (Sumsub flow, ID + selfie, <5 min), and what to do on failure.
    • Updated ## Getting Started intro and step 2 to lead with the new gate.
    • Updated the :::danger callout from "you can apply without KYC" to "you can't apply without KYC."
  • wave/points-and-rewards.mdx
    • Reworded the "Requirements to Receive Funds" paragraph and the :::danger callout to reflect that KYC happens at application time.
  • wave/contributors/faq.mdx
    • "Do I need to pay to participate?" answer now mentions the KYC-at-application gate alongside the withdrawal wallet requirement.

Untouched

  • wave/withdrawing-rewards.mdx — still accurate (withdrawal-time KYC requirement is unchanged).
  • wave/terms-and-rules.mdx — legal language about KYC as a prerequisite for reward allocation still holds.

Test plan

  • Build the site locally and confirm #verifying-your-identity is reachable from solving-issues-and-earning-rewards.
  • Confirm the cross-page links from points-and-rewards and faq resolve.

KYC is now required to apply for any issue, not just to withdraw rewards.
Update contributor docs accordingly and add a dedicated "Verifying your
identity" section that the app's KYC-required prompt links to.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for frosty-shirley-cac2f7 failed. Why did it fail? →

Name Link
🔨 Latest commit 4706440
🔍 Latest deploy log https://app.netlify.com/projects/frosty-shirley-cac2f7/deploys/6a0ec17ae4477c0008fb8b00

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Wave contributor documentation to reflect the new requirement that users must complete identity verification (KYC) before applying to issues, and adds a dedicated “Verifying your identity” anchor that the app can link to.

Changes:

  • Added a new “## Verifying your identity” section (and anchor) with KYC rationale, steps, and failure handling.
  • Updated multiple pages’ copy/callouts to reflect KYC being required at application time (not only at withdrawal time).
  • Updated the Settings navigation path for completing KYC to Settings → Profile.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
docs/pages/wave/contributors/solving-issues-and-earning-rewards.mdx Adds the new KYC section/anchor and updates “Getting Started” + callout to reflect the application-time gate.
docs/pages/wave/points-and-rewards.mdx Rewords withdrawal requirements and the KYC callout to align with KYC being required before applying.
docs/pages/wave/contributors/faq.mdx Updates participation requirements and adds an FAQ explaining why KYC is required before applying.
docs/pages/wave/withdrawing-rewards.mdx Updates the Settings navigation path for where to complete KYC in the app UI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/pages/wave/withdrawing-rewards.mdx
@efstajas efstajas merged commit a0ef985 into main May 21, 2026
4 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants